Should we follow a pattern of making these class methods instead of instance methods? The calling code looked weird to me.

the advantage of using instance is you get the initialize method to do some setup first -- but it's definitely possible to manage this with class or module (eg, singleton instance on class to do the work)

It seems more Railsy to do it the singleton way, ie User.first etc etc

converted these to class-method interfaces